Skip to content

[dice] Refactor some dice related constants and APIs #25023

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Nov 8, 2024

Conversation

tommychiu-github
Copy link
Contributor

Some DICE key creation constants and utilities are invoked by several modules.
Refactor them for a better code reuse.

@tommychiu-github tommychiu-github requested a review from a team as a code owner November 6, 2024 04:19
@tommychiu-github tommychiu-github requested review from pamaury and removed request for a team November 6, 2024 04:19
@tommychiu-github
Copy link
Contributor Author

@timothytrippel for viz.

@tommychiu-github tommychiu-github force-pushed the main branch 2 times, most recently from 3567df4 to 6c60a5c Compare November 6, 2024 05:21
@timothytrippel timothytrippel self-requested a review November 7, 2024 18:32
It'll be invoked by several modules, so move it to base:util library for
code reuse.

Signed-off-by: Tommy Chiu <[email protected]>
Both X509 and CWT DICE implementations need these definitions. Move it
out from the original source file for code reuse.

Signed-off-by: Tommy Chiu <[email protected]>
The autogen source from templates needs some more APIs to utilize the
Cbor structure, including
- calculate the size of a given cbor argument
- add a bstr/tstr header with size, and rewind the cursor
- copy the raw data from input pointer to the CborOut buffer

Signed-off-by: Tommy Chiu <[email protected]>
@timothytrippel timothytrippel merged commit d8af8fb into lowRISC:master Nov 8, 2024
41 checks passed
@timothytrippel timothytrippel added the CherryPick:earlgrey_1.0.0 This PR should be cherry-picked to earlgrey_1.0.0 label Nov 8, 2024
Copy link

github-actions bot commented Nov 8, 2024

Successfully created backport PR for earlgrey_1.0.0:

Copy link

github-actions bot commented Nov 8, 2024

Git push to origin failed for earlgrey_1.0.0 with exitcode 1

@github-actions github-actions bot added the Manually CherryPick This PR should be manually cherry picked. label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPick:earlgrey_1.0.0 This PR should be cherry-picked to earlgrey_1.0.0 Manually CherryPick This PR should be manually cherry picked.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants